Skip to content

Minibwa speedup - #325

Open
ignacio3437 wants to merge 13 commits into
Plant-Food-Research-Open:devfrom
ignacio3437:minibwa
Open

Minibwa speedup#325
ignacio3437 wants to merge 13 commits into
Plant-Food-Research-Open:devfrom
ignacio3437:minibwa

Conversation

@ignacio3437

@ignacio3437 ignacio3437 commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Updating bwa-index and bwa-mem to minibwa-index and minibwa-map.

Running the full HY hic dataset results in significant speedups from 11 H 10min to 6 H 20 min to run the full pipeline. The heatmaps look idential although the relutling bam files are not identical as evident from the hicqc.

I am having major issues with the nf-core linting steps even though I installed the module using nf-core modules install. Will need some help there to get the pipeline to lint, but it seems to run fine. I'm also not able to run the tests to update the snapshots on my WSL laptop. Is it possible to do this on PP?

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes: nextflow run . -profile test,docker --outdir <OUTDIR> and nf-test test --profile docker tests/.
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@github-actions

Copy link
Copy Markdown

This PR is against the main branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @ignacio3437,

It looks like this pull-request is has been made against the ignacio3437/assemblyqc main branch.
The main branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to main are only allowed if they come from the ignacio3437/assemblyqc dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@ignacio3437

Copy link
Copy Markdown
Collaborator Author

See results here:
MiniBWA_Tests.pdf

@ignacio3437
ignacio3437 changed the base branch from main to dev July 22, 2026 03:05
@GallVp

GallVp commented Jul 22, 2026

Copy link
Copy Markdown
Member

@ignacio3437

Great work. That's a big improvement. Let me take care of the merge.

'https://depot.galaxyproject.org/singularity/haphic:1.0.7--hdfd78af_0':
'biocontainers/haphic:1.0.7--hdfd78af_0' }"
'docker://gallvp/haphic:1.0.7--hdfd78af_0':
'docker.io/gallvp/haphic:1.0.7--hdfd78af_0' }"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

container 'docker.io/gallvp/haphic:1.0.7--hdfd78af_0'

This is adequate for both singularity and docker.

@GallVp

GallVp commented Jul 23, 2026

Copy link
Copy Markdown
Member

I'll get to it after GallVp/nxf-components#104

The upstream modules and sub workflows need an update. The modified sub workflow comes from nxf-components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants